Skip to content

Add MCP server support for Consumption Logic Apps#8901

Open
Bhavd13 wants to merge 20 commits intomainfrom
bhavya/mcpconsumption
Open

Add MCP server support for Consumption Logic Apps#8901
Bhavd13 wants to merge 20 commits intomainfrom
bhavya/mcpconsumption

Conversation

@Bhavd13
Copy link
Contributor

@Bhavd13 Bhavd13 commented Mar 9, 2026

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

Adds MCP (Model Context Protocol) server support for Consumption Logic Apps. This enables consumption SKU workflows to connect to custom MCP servers and use MCP tools as actions, bringing feature parity with the Standard SKU's existing MCP support.

Impact of Change

  • Users: Consumption Logic Apps users can now add MCP Client connections (custom servers with 7 auth types) and use MCP tool actions in their workflows.
  • Developers: New consumption-specific MCP connector manifest, operation manifest, and connection creation flow. Follows existing patterns from Standard MCP implementation.
  • System: No backend changes — frontend only

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in:

Contributors

@Bhavd13

Screenshots/Videos

mcp.recording.mp4

Copilot AI review requested due to automatic review settings March 9, 2026 08:32
@github-actions
Copy link

github-actions bot commented Mar 9, 2026

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

PR Title

  • Current: Add MCP server support for Consumption Logic Apps
  • Issue: None — title is clear and describes the change.
  • Recommendation: Title is good. (Optional) If you want stronger scoping, you can prefix with a scope/tag, e.g. Consumption: Add MCP server support for Consumption Logic Apps or designer(consumption): Add MCP server support.

Commit Type

  • Properly selected (feature)
  • Only one commit type is selected, which is correct.

Risk Level

  • The PR body marks the change as Medium and the PR has a risk:medium label. These match.
  • Assessment based on the diff (20 commits, ~1632 additions, 68 deletions across 15 files) indicates medium risk is appropriate: this introduces new client-side connection/manifest/serializer logic and unit tests but does not touch backend code.

What & Why

  • Current: Adds MCP server support for Consumption Logic Apps. Enables consumption SKU workflows to connect to custom MCP servers and use MCP tools as actions.
  • Issue: None — concise and clear.
  • Recommendation: Optionally mention any feature-flag or rollout plan if applicable, or list any known compatibility constraints for older workflows.

Impact of Change

  • The PR body contains Users/Developers/System sections and explains impacts.
  • Recommendation: Small suggestions to improve the section (optional):
    • Users: call out whether existing consumption workflows remain unchanged unless they opt-in to MCP tools.
    • Developers: add a pointer to where the new manifests/services live (e.g., libs/logic-apps-shared/src/designer-client-services/lib/consumption).
    • System: you already state "frontend only" — ensure any Ops/QA teams are aware if new client-side behavior requires test environments that host MCP servers for integration tests.

Test Plan

  • Unit tests added/updated: Confirmed. I see multiple new/updated test files under libs/logic-apps-shared/src/designer-client-services/lib/consumption/tests and other tests covering Connection/Connector/OperationManifest/ConnectionService logic.
  • E2E tests: not added. This is acceptable but consider adding at least one E2E/integration test or manual verification steps that exercise a real MCP server or a mocked server path to validate end-to-end behavior.
  • Manual testing: not selected — fine given unit tests, but please add manual test steps if reviewers expect hands-on verification of the new connection flows.

Contributors

  • Contributors listed: @Bhavd13 — good. Remember to credit PMs/Designers/other reviewers if they contributed.

⚠️ Screenshots/Videos

  • A link to an asset is present in the PR body.
  • Recommendation: since this changes connection creation UI and the operation input flow, please attach screenshots or a short video (1-2 minutes) showing the new MCP connection creation and adding an MCP tool action — this helps reviewers verify UX changes quickly.

Summary Table

Section Status Recommendation
Title Keep as-is or optionally add a scope prefix
Commit Type None
Risk Level Medium is appropriate given the surface area
What & Why Consider adding rollout/compat notes
Impact of Change Add brief pointer to new files and confirm UX expectations
Test Plan Consider adding an E2E or integration test and/or manual test steps
Contributors None
Screenshots/Videos ⚠️ Add screenshots or short video of UI flows

Final Message
This PR passes the PR title/body checks. The advised risk level from the diff is risk:medium, which matches the submitter's declared risk. Good job including unit tests — I confirmed test files in the diff. Please consider the optional items above: add a short note about rollout/migration if relevant, include a short E2E/integration test or manual test steps that exercise MCP end-to-end, and attach screenshots or a short video showing the UX flow for creating an MCP connection and using an MCP tool. Once those (optional) items are added, this will be easier for reviewers to validate end-to-end behavior. Thank you!


Last updated: Mon, 16 Mar 2026 18:42:06 GMT

@Bhavd13 Bhavd13 added risk:medium Medium risk change with potential impact and removed needs-pr-update labels Mar 9, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Consumption SKU support for MCP (Model Context Protocol) by introducing a consumption MCP connector + built-in operation manifest wiring, and updating the connection creation UI/service to recognize MCP connections.

Changes:

  • Add consumption MCP connector + built-in MCP Client operation manifest and wire them into the consumption operation manifest service.
  • Extend ConsumptionConnectionService to recognize MCP connectors and create MCP connections.
  • Update Designer V2 create-connection panel behavior for consumption MCP connections and add unit tests for the new connection logic.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
package.json Tweaks lint-staged ESLint invocation to reduce ignored-file warnings.
libs/logic-apps-shared/src/designer-client-services/lib/consumption/operationmanifest.ts Adds MCP built-in operation support (mcpclienttoolnativemcpclient) and returns the MCP operation manifest/operation metadata.
libs/logic-apps-shared/src/designer-client-services/lib/consumption/manifests/mcpclientconnector.ts Introduces the consumption MCP connector manifest (auth parameter sets, metadata).
libs/logic-apps-shared/src/designer-client-services/lib/consumption/manifests/builtinmcpclient.ts Adds the built-in MCP Client operation manifest including listMcpTools dynamic list for tool selection.
libs/logic-apps-shared/src/designer-client-services/lib/consumption/connection.ts Adds MCP connector lookup and MCP-specific connection creation paths (built-in vs managed).
libs/logic-apps-shared/src/designer-client-services/lib/consumption/tests/connection.spec.ts Adds unit tests for MCP connection creation helpers/paths.
libs/designer-v2/src/lib/ui/panel/connectionsPanel/createConnection/createConnection.tsx Hides the “name” input for Consumption MCP connections via a runtime service check.

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

📊 Coverage check completed. See workflow run for details.

@Bhavd13 Bhavd13 deployed to E2ETesting March 16, 2026 18:44 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-validated risk:medium Medium risk change with potential impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants